Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: Add Shift State Sensor #569

Merged
merged 4 commits into from
Apr 21, 2023
Merged

Conversation

Megabytemb
Copy link
Contributor

Add a car Shift state ENUM sensor, so you can detect what gear the car is in specifically.

This is different to the Parking Break binary sensor, as you get the explicit gear.

@alandtse
Copy link
Owner

Wait, isn't an enum an input class in HA?

@Megabytemb
Copy link
Contributor Author

I got it from the Sensor Dev site

https://developers.home-assistant.io/docs/core/entity/sensor/

"
The sensor has a limited set of (non-numeric) states. The options property must be set to a list of possible states when using this device class.
"

@alandtse
Copy link
Owner

Ok, does our minimum version of ha cover this sensor or would we need to increment it. It's the hacs.json.

@Megabytemb
Copy link
Contributor Author

Looks like it was introduced in 2023.1.0

shall we bump the min version required? or remove the fancy enum stuff and just use a generic sensor?

@alandtse
Copy link
Owner

We can bump.

@alandtse
Copy link
Owner

Oh, can you please make the bump so I can merge this in?

@Megabytemb
Copy link
Contributor Author

Bumped :)

@alandtse
Copy link
Owner

Thanks. But I thought you said it came in in 2023.1? The way to tell is finding the commit that brought it in.

@Megabytemb
Copy link
Contributor Author

Right you are. Fixed

@alandtse alandtse merged commit 72ac435 into alandtse:dev Apr 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants